feat(source-set): add dependency-aware build order - #50

tests/cli_build.rs (3)

31-46: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Дублирование форматной строки скрипта.

write_recording_build_script повторяет тело write_build_script (строки 15–29) целиком, отличаясь одной строкой записи в лог. При изменении поведения фейкового 1cv8 придётся править два места.

♻️ Вариант объединения helper'ов

-fn write_recording_build_script(path: &Path, calls_log: &Path, fail_pattern: Option<&str>) {
-    let pattern_branch = fail_pattern
-        .map(|pattern| {
-            format!(
-                "if printf '%s' \"$args\" | grep -F -q -- '{}'; then exit 17; fi",
-                pattern
-            )
-        })
-        .unwrap_or_default();
-    let body = format!(
-        "args=\"$*\"\nout=\"\"\nprev=\"\"\nfor arg in \"$@\"; do\n  if [ \"$prev\" = \"/Out\" ]; then out=\"$arg\"; fi\n  prev=\"$arg\"\ndone\nprintf '%s\\n' \"$args\" >> '{}'\nif [ -n \"$out\" ]; then printf 'designer log for %s\\n' \"$args\" > \"$out\"; fi\n{}\nexit 0",
-        calls_log.display(),
-        pattern_branch
-    );
-    write_script(path, &body);
-}
+fn write_recording_build_script(path: &Path, calls_log: &Path, fail_pattern: Option<&str>) {
+    write_build_script_with_log(path, Some(calls_log), fail_pattern);
+}

write_build_script при этом становится write_build_script_with_log(path, None, fail_pattern), а строка записи в лог добавляется в тело только при Some(calls_log).

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@tests/cli_build.rs` around lines 31 - 46, Устраните дублирование между
write_recording_build_script и write_build_script: выделите общее формирование
тела в write_build_script_with_log, принимающий опциональный calls_log и
fail_pattern. Добавляйте строку записи аргументов только при Some(calls_log), а
существующий write_build_script переведите на этот helper с None, сохранив
текущее поведение обоих сценариев.

---

702-715: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Общий корень: нет общего helper'а для разбора лога вызовов /UpdateDBCfg. Правило «строка с -Extension <name> относится к расширению, иначе к конфигурации» скопировано в три места; при изменении формата аргументов придётся править все.

• tests/cli_build.rs#L702-L715: заменить inline-блок вызовом общего helper'а (например update_db_cfg_order) из tests/support/mod.rs.
• tests/cli_build.rs#L771-L784: заменить второй inline-блок тем же helper'ом.
• tests/cli_test.rs#L479-L494: использовать тот же helper, вынеся ветвь RunUnitTests= → enterprise в параметр или отдельную обёртку.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@tests/cli_build.rs` around lines 702 - 715, Extract the duplicated
/UpdateDBCfg call-log parsing into a shared helper in tests/support/mod.rs, such
as update_db_cfg_order, preserving the rule that -Extension <name> maps to that
extension and other calls map to main. Replace both inline parsing blocks in
tests/cli_build.rs at lines 702-715 and 771-784 with the helper, and update
tests/cli_test.rs at lines 479-494 to use it while supporting the RunUnitTests=
→ enterprise distinction through a parameter or dedicated wrapper.

---

800-804: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Магический код выхода и хрупкий шаблон фейла.

Some(4) без пояснения затрудняет чтение — стоит сослаться на константу/enum кода ошибки, как это сделано в src/domain/test.rs. Кроме того, fail_pattern требует, чтобы /UpdateDBCfg и -Extension yaxunit шли строго подряд в $*; любое добавление аргумента между ними тихо изменит сценарий теста (сборка перестанет падать). Рассмотрите два отдельных grep по подстрокам вместо одной склеенной.

Также setup_dependency_project уже записывает скрипт, а строки 800–804 сразу его перезаписывают — можно параметризовать helper fail_pattern.

Also applies to: 819-820

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@tests/cli_build.rs` around lines 800 - 804, Update the build-script test
setup around setup_dependency_project and write_recording_build_script to accept
fail_pattern as a helper parameter instead of writing the script again at the
call sites. Replace the magic exit status Some(4) with the existing named
error-code constant or enum used by the test domain. Make fail_pattern validate
/UpdateDBCfg and -Extension yaxunit independently so inserted arguments do not
alter the intended failure scenario.

tests/cli_test.rs (1)

171-178: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Таймаут зашит в helper и дублирует аргумент setup_project.

execution_timeout_seconds: 5 жёстко прописан, хотя тот же 5 передаётся в setup_project (строка 432). При правке одного значения второе тихо разойдётся. Стоит принять таймаут параметром.

♻️ Предлагаемая правка

-fn write_dependency_test_config(path: &Path, work_path: &Path, install_dir: &Path) {
+fn write_dependency_test_config(
+    path: &Path,
+    work_path: &Path,
+    install_dir: &Path,
+    timeout_seconds: u64,
+) {
     let config = format!(
-        "workPath: '{}'\n...\n  execution_timeout_seconds: 5\n...",
+        "workPath: '{}'\n...\n  execution_timeout_seconds: {}\n...",
         work_path.display(),
+        timeout_seconds,
         install_dir.display(),
     );

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@tests/cli_test.rs` around lines 171 - 178, Update
write_dependency_test_config to accept an execution-timeout parameter and
interpolate it into the generated configuration instead of hardcoding 5. Pass
the existing timeout value from setup_project at its call site, keeping the
configuration and setup argument synchronized.

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes